-
Notifications
You must be signed in to change notification settings - Fork 558
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merge the codelabs branch into main branch #1823
Conversation
* initial setup * setup YAML parsing * format * load codelab files * move logic into CodelabUi class, start stubbing out code * Set up right panels * Add CodeMirror editor * Add CodelabState * fix HTML * Add step buttons * remove print * Load code and update when the step changes * Add run button and console panel * add flutter_codelab * Ignore /example/codelabs directory in analysis and builders * Add tabs to console panel * Get flutter compilation working * add TODO * exclude codelabs in dart_source_cleanup build step * remove whitespace
* Add GithubCodelabFetcher * make gh_ref and gh_path optional query parameters
* add codelabs.html to allowed files in main.py * Add show solution button
FYI the only unreviewed commit is 4a90fa3, which runs |
@kevmoo should I commit the
But 3353d19 fails because it does analysis before building the project. What's the best-practice here? |
Should have |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Trailing LGTM
No description provided.